Skip to content
This repository was archived by the owner on Feb 14, 2019. It is now read-only.

Conversation

@jgutix
Copy link

@jgutix jgutix commented Jan 12, 2017

The update method will compare viewDate instead of date to correctly reset(or not) the viewDate attribute, viewDate will reset to date on the hide method. Fixes #617

…esetting the viewDate, this will prevent to reset the view of the month if the view already moved away from the current selected date.
@AuspeXeu
Copy link
Collaborator

Please fix the tests before we can merge this.

…e` call within `setDatesDisabled` otherwise it will reset the `viewDate` (aka issue smalot#617)
@jgutix
Copy link
Author

jgutix commented Nov 15, 2017

@AuspeXeu here is a refactored fix for the issue #617 my previous "solution" had the side effect of preventing the date to be updated by code, that's why the tests were failing. This solution makes the tests pass without problem, I'm also adding a new test specifically for the setDatesDisabled functionality.
Also the tests were not running properly with the latest version of phamtomjs, which is pulled by travis, so I'm adding a fix for that too.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants